Skip to content

Remove /v1 from all API endpoints #3

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
blocksorg wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Remove /v1 from all API endpoints #3

blocksorg wants to merge 1 commit into from

Conversation

@blocksorg
Copy link

@blocksorg blocksorg bot commented Oct 27, 2025

Summary

  • Removed /v1 prefix from all API endpoint references across the codebase
  • Updated billing/redeem_credits endpoint in login scripts (Python and TypeScript)
  • Updated mock test endpoints from /v1/responses to /responses
  • Updated API endpoint comments and documentation
  • Updated model provider info comments

Changes

  • codex-rs/login/src/login_with_chatgpt.py: Updated billing endpoint from /v1/billing/redeem_credits to /billing/redeem_credits
  • codex-cli/src/utils/get-api-key.tsx: Updated billing endpoint from /v1/billing/redeem_credits to /billing/redeem_credits
  • codex-rs/core/tests/stream_no_completed.rs: Updated mock endpoint from /v1/responses to /responses
  • codex-rs/core/tests/previous_response_id.rs: Updated mock endpoints from /v1/responses to /responses
  • codex-rs/core/tests/live_cli.rs: Updated comment reference from /v1/responses to /responses
  • codex-rs/core/tests/live_agent.rs: Updated comment reference from /v1/responses to /responses
  • codex-rs/core/src/models.rs: Updated comment reference from /v1/responses to /responses
  • codex-rs/core/src/model_provider_info.rs: Updated comments from /v1/responses and /v1/chat/completions to /responses and /chat/completions

Impact

This change simplifies the API structure by removing version prefixes from all endpoints.

Test plan

  • Verify all unit tests pass
  • Verify mock API tests continue to work with updated endpoints
  • Verify billing credit redemption continues to work (if applicable)

🤖 Generated with Claude Code

@claude
Remove /v1 from all API endpoints
36df5a6 
Updated all endpoint references to remove /v1 prefix:
- Updated billing/redeem_credits endpoint in login scripts
- Updated mock test endpoints from /v1/responses to /responses
- Updated API endpoint comments and documentation
- Updated model provider info comments

This aligns with the new API structure without version prefixes.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@blocksorg
Copy link
Author

blocksorg bot commented Oct 27, 2025

Mention Blocks like a regular teammate with your question or request:

@blocks review this pull request
@blocks make the following changes ...
@blocks create an issue from what was mentioned in the following comment ...
@blocks explain the following code ...
@blocks are there any security or performance concerns?

Powered by Blocks

@cubeorgdev
Copy link

cubeorgdev bot commented Oct 27, 2025

Mention Cube like a regular teammate with your question or request:

@cube review this pull request
@cube make the following changes ...
@cube create an issue from what was mentioned in the following comment ...
@cube explain the following code ...
@cube are there any security or performance concerns?

Powered by Blocks | 📚 Documentation

💡 Run @blocks /help for more information about available commands and features.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant